#285 Исправлена ошибка в проверяющем тесте в уроке Неизменяемость строк #296
Conversation
|
Добрый день @alepekhin тесты падают. Здесь ведь мы сравниваем со строкой, а в ней нет переноса. |
|
Исправлено. Проблема в том, что в приложении мы используем System.out.println() которое добавляет '\n' в конец вывода, в то время как мы ожидаем текст без этого символа |
|
Переписал еще раз немного поменяв постановку. Чтобы не путаться с print и println лучше попросить написать метод, который вернет нормализованный email |
| return email.trim().toLowerCase(); | ||
| // END | ||
| } | ||
|
|
There was a problem hiding this comment.
емнип на данном этапе студенты не работали со свойствами. Поэтому давайте оставим все внутри одного метода, с переменной. Да и создание своих методов рассмаривается дальше (поэтому мы тут и печатаем на экран).
There was a problem hiding this comment.
Хорошо. Будем печатать. Печать добавляет перевод строки, который мы здесь, как и в других тестах, удаляем с помощью actual.trim() Но это слишком сильно, так как удаляет не только последний символ. Из за этого получился данный баг. Теперь в тесте удаляется только последний символ, и только если он - перевод строки
| email = email.trim(); | ||
| email = email.toLowerCase(); | ||
| System.out.println(email); | ||
| System.out.println(email.trim().toLowerCase()); |
There was a problem hiding this comment.
Так, а теперь получается нет разницы с тем, что было раньше.
Только если обратите внимание, то мы ранее перезаписывали еременную, тк цепочку методов изучаем в след уроке.
| public static String normalize() { | ||
| ... | ||
| } | ||
| ``` |
There was a problem hiding this comment.
Здесь осталось про метод normalize
There was a problem hiding this comment.
Вернул обратно как было. Да, теперь получается нет разницы с тем, что было раньше, за исключением теста. Причина бага как раз в нем
2 participants
Мы не должны менять actual перед срвнением с expected